First Jekyll version of the ipa-lab github page.#1
Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates the IPA-Lab website from a static HTML site to a Jekyll-based static site generator setup. The migration modernizes the site architecture by introducing templating, data-driven content generation, and improved maintainability through separation of content and presentation.
Changes:
- Removed old static HTML files and custom CSS in favor of Jekyll templates and Bootstrap
- Added Jekyll configuration, layouts, and includes for modular site structure
- Introduced Python scripts to auto-generate team and publication data from external APIs (TISS and DBLP)
- Set up GitHub Actions workflow for automated Jekyll site deployment
Reviewed changes
Copilot reviewed 20 out of 24 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| index.html → index.md | Migrated home page from static HTML to Markdown with Jekyll front matter; fixed duplicate "we" typo |
| styles/tuwien.css (deleted) | Removed old custom CSS in favor of Bootstrap and simplified main.css |
| css/main.css | Added minimal custom CSS for navbar spacing and card hover effects |
| _config.yml | Added Jekyll configuration with site metadata, plugins, and collections setup |
| _layouts/default.html | Created main layout template with header, content area, and footer |
| _includes/head.html | Created head partial with Bootstrap CDN links and metadata |
| _includes/header.html | Created navigation header with Bootstrap navbar |
| _includes/footer.html | Created footer with copyright, links, and contact info |
| _pages/*.md | Added Markdown content pages for team, publications, research, vacancies, and 404 |
| _scripts/fetch_tiss.py | Added Python script to fetch team data from TU Wien TISS API |
| _scripts/fetch_dblp.py | Added Python script to fetch publication data from DBLP API |
| _data/team/tiss_generated.yml | Generated team member data from TISS API |
| _data/publications/dblp_generated.yml | Generated publication list from DBLP API |
| .github/workflows/jekyll-gh-pages.yml | Added GitHub Actions workflow for automated site deployment |
| Gemfile & Gemfile.lock | Added Ruby dependencies for Jekyll and plugins |
| .gitignore | Added Jekyll-specific ignored files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
_scripts/fetch_tiss.py
Outdated
| for tiss_id in tiss_ids: | ||
| url = f"https://tiss.tuwien.ac.at/api/person/v23/id/{tiss_id}" | ||
| print(f"Fetching TISS ID {tiss_id}...") | ||
| resp = requests.get(url) |
There was a problem hiding this comment.
The script makes HTTP requests without timeout parameters, which could cause the script to hang indefinitely if the remote server doesn't respond. Consider adding a timeout parameter to the requests.get() call, for example: requests.get(url, timeout=10).
_scripts/fetch_dblp.py
Outdated
| for pid in dblp_ids: | ||
| url = f"https://dblp.org/pid/{pid}.xml" | ||
| print(f"Fetching DBLP data for {pid}...") | ||
| resp = requests.get(url) |
There was a problem hiding this comment.
The script makes HTTP requests without timeout parameters, which could cause the script to hang indefinitely if the remote server doesn't respond. Consider adding a timeout parameter to the requests.get() call, for example: requests.get(url, timeout=10).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…overviews Co-authored-by: JakobHoffmann <23260378+JakobHoffmann@users.noreply.github.com>
Add README documenting Jekyll site setup and Python scripts
No description provided.